Skip to content

Conversation

@Laure-di
Copy link
Contributor

No description provided.

@Laure-di Laure-di requested a review from a team as a code owner November 26, 2025 15:50
@Laure-di Laure-di force-pushed the baremetal-cloud-init branch from 847763a to d06a108 Compare November 26, 2025 15:52
@Laure-di Laure-di requested a review from remyleone as a code owner November 26, 2025 15:53
@remyleone remyleone marked this pull request as draft November 26, 2025 17:31
@github-actions github-actions bot added rdb Managed MySQL and PostgreSQL issues, bugs and feature requests instance Instance issues, bugs and feature requests iam IAM issues, bugs and feature requests labels Nov 27, 2025
@Laure-di Laure-di force-pushed the baremetal-cloud-init branch 2 times, most recently from 234e82e to e72905d Compare November 27, 2025 10:40
@Laure-di Laure-di force-pushed the baremetal-cloud-init branch from 12b1c42 to 269a444 Compare November 27, 2025 13:40
@codecov-commenter
Copy link

codecov-commenter commented Nov 27, 2025

Codecov Report

❌ Patch coverage is 0% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 1.76%. Comparing base (b919f45) to head (ce21317).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
internal/services/baremetal/server.go 0.00% 12 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##           master   #3478      +/-   ##
=========================================
- Coverage    1.76%   1.76%   -0.01%     
=========================================
  Files         405     405              
  Lines       44318   44330      +12     
=========================================
  Hits          784     784              
- Misses      43446   43458      +12     
  Partials       88      88              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Laure-di Laure-di force-pushed the baremetal-cloud-init branch 2 times, most recently from c97cd6c to 5fd408a Compare November 30, 2025 14:36
@Laure-di Laure-di marked this pull request as ready for review November 30, 2025 15:05
@Laure-di Laure-di force-pushed the baremetal-cloud-init branch from 5fd408a to ddccb0a Compare December 1, 2025 08:32
@Laure-di Laure-di force-pushed the baremetal-cloud-init branch from ddccb0a to ce21317 Compare December 1, 2025 08:33
resource "scaleway_baremetal_server" "my_server_ci" {
zone = "fr-par-2"
offer = data.scaleway_baremetal_offer.my_offer.offer_id
os = "d17d6872-0412-45d9-a198-af82c34d3c5c"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use a data source and no hard coded values

- `ipam_ip_ids` - (Optional) List of IPAM IP IDs to assign to the server in the requested private network.
- `zone` - (Defaults to [provider](../index.md#zone) `zone`) The [zone](../guides/regions_and_zones.md#zones) in which the server should be created.
- `partitioning` (Optional) The partitioning schema in JSON format
- `cloud_init` - (Optional) Configuration data to pass to cloud-init such as a YAML cloud config or a user-data script. Accepts either a string containing the content or a path to a file (for example `file("cloud-init.yml")`). Max length: 127998 characters. Updates to `cloud_init` will update the server user-data via the API and do not trigger a reinstall; however, a reboot of the server is required for the OS to re-run cloud-init and apply the changes. Only supported for OSes that have cloud-init enabled.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a way to see if cloud init is supported or not?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

baremetal iam IAM issues, bugs and feature requests instance Instance issues, bugs and feature requests rdb Managed MySQL and PostgreSQL issues, bugs and feature requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants